Skip to content

chore: upgrade Greenwood v0.33.0#211

Merged
thescientist13 merged 15 commits intomainfrom
chore/upgrade-greenwood-v0.33.0
Sep 14, 2025
Merged

chore: upgrade Greenwood v0.33.0#211
thescientist13 merged 15 commits intomainfrom
chore/upgrade-greenwood-v0.33.0

Conversation

@thescientist13
Copy link
Copy Markdown
Member

@thescientist13 thescientist13 commented May 25, 2025

Related Issue

ProjectEvergreen/greenwood#1499

Summary of Changes

  1. Upgrade Greenwood to v0.33.0

TODO

  1. validate / absorb document bare specifier CSS @imports #214
  2. active frontmatter title not merging as expected - bug/issue 1475 reconcile page level custom title and layout <title> merging when using active content greenwood#1526
  3. getting a build error in the CSS Modules plugin (again...) - bug(plugins): issue 823 refactor CSS Modules plugin for better concurrency safety greenwood#1561
  4. copy assets broken - bug(cli): issue 823 safely copy files with their directories after concurrency refactor greenwood#1558

Not sure if this is also related to the concurrency refactor, but I think we need more ThreadPools or something?

(node:85019) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 Symbol(kWorkerFreedEvent) listeners added to [WorkerPool]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)

@netlify
Copy link
Copy Markdown

netlify Bot commented May 25, 2025

Deploy Preview for super-tapioca-5987ce ready!

Name Link
🔨 Latest commit 4d0afbf
🔍 Latest deploy log https://app.netlify.com/projects/super-tapioca-5987ce/deploys/68c6d0fa330dd1000815b64d
😎 Deploy Preview https://deploy-preview-211--super-tapioca-5987ce.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@thescientist13

This comment was marked as resolved.

@thescientist13 thescientist13 added the needs upstream Notes that this item exposes an issue with a dependency (usually Greenwood) label Jun 28, 2025
@thescientist13 thescientist13 removed the needs upstream Notes that this item exposes an issue with a dependency (usually Greenwood) label Jul 5, 2025
@thescientist13 thescientist13 force-pushed the chore/upgrade-greenwood-v0.33.0 branch from d1f6010 to 9d508a7 Compare July 5, 2025 23:46
@thescientist13 thescientist13 added the needs upstream Notes that this item exposes an issue with a dependency (usually Greenwood) label Aug 23, 2025
@thescientist13
Copy link
Copy Markdown
Member Author

getting a build error in the CSS Modules plugin (again...)

SyntaxError [Error]: Unterminated string in JSON at position 8192 (line 1 column 8193)
  at JSON.parse (<anonymous>)
  at getCssModulesMap (file:///Users/owenbuckley/Workspace/project-evergreen/www.greenwoodjs.dev/node_modules/@greenwood/plugin-css-modules/src/index.js:29:26)
  at ScanForCssModulesResource.shouldIntercept (file:///Users/owenbuckley/Workspace/project-evergreen/www.greenwoodjs.dev/node_modules/@greenwood/plugin-css-modules/src/index.js:194:27)
  at getCustomLoaderResponse (file:///Users/owenbuckley/Workspace/project-evergreen/www.greenwoodjs.dev/node_modules/@greenwood/cli/src/loader.js:71:49)
  at async load (file:///Users/owenbuckley/Workspace/project-evergreen/www.greenwoodjs.dev/node_modules/@greenwood/cli/src/loader.js:113:28)
  at async nextLoad (node:internal/modules/esm/hooks:748:22)
  at async Hooks.load (node:internal/modules/esm/hooks:385:20)
  at async handleMessage (node:internal/modules/esm/worker:199:18)

@thescientist13 thescientist13 added enhancement New feature or request and removed chore project maintenance related tasks labels Aug 23, 2025
@thescientist13
Copy link
Copy Markdown
Member Author

On top of the CSS Modules issue, this project will also suffer from ProjectEvergreen/greenwood#1558

@thescientist13 thescientist13 changed the title chore/upgrade Greenwood v0.33.0 chore: upgrade Greenwood v0.33.0 Sep 1, 2025
@thescientist13 thescientist13 removed the needs upstream Notes that this item exposes an issue with a dependency (usually Greenwood) label Sep 1, 2025
@thescientist13 thescientist13 force-pushed the chore/upgrade-greenwood-v0.33.0 branch from 2865b0b to a6f9312 Compare September 6, 2025 18:42
@thescientist13 thescientist13 marked this pull request as ready for review September 6, 2025 18:43
@thescientist13 thescientist13 moved this from 🏗 In progress to 👀 In review in [Greenwood] Phase 10 - Ecosystem Compat Sep 6, 2025
@thescientist13
Copy link
Copy Markdown
Member Author

Not sure if this is also related to the concurrency refactor, but I think we need more ThreadPools or something?

(node:85019) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 Symbol(kWorkerFreedEvent) listeners added to [WorkerPool]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)

@thescientist13 thescientist13 force-pushed the chore/upgrade-greenwood-v0.33.0 branch from c0b96cb to 4d0afbf Compare September 14, 2025 14:28
@thescientist13 thescientist13 merged commit f96c9f6 into main Sep 14, 2025
5 checks passed
@thescientist13 thescientist13 deleted the chore/upgrade-greenwood-v0.33.0 branch September 14, 2025 14:31
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in [Greenwood] Phase 10 - Ecosystem Compat Sep 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request release/v0.33.0

Projects

Development

Successfully merging this pull request may close these issues.

1 participant